Skip to content

Conversation

@dmlvr
Copy link
Contributor

@dmlvr dmlvr commented Jan 9, 2026

No description provided.

@dmlvr dmlvr self-assigned this Jan 9, 2026
@dmlvr dmlvr added the 26_1 label Jan 9, 2026
@dmlvr dmlvr marked this pull request as ready for review January 12, 2026 10:08
@dmlvr dmlvr requested a review from a team as a code owner January 12, 2026 10:08
Copilot AI review requested due to automatic review settings January 12, 2026 10:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes TypeScript problems in React demos for VectorMap, FloatingActionButton, and Localization components, enabling the removal of exclusions from tsconfig.react-check.json.

Changes:

  • Added TypeScript type definitions for VectorMap data modules and improved type safety across VectorMap demos
  • Enhanced TypeScript types in FloatingActionButton demo with proper ref typing and data structure definitions
  • Added global TypeScript declarations for Globalize and JSON imports in Localization demo
  • Changed tooltip functions to return empty objects instead of null for consistency
  • Added proper ref types and optional chaining for component instance access

Reviewed changes

Copilot reviewed 36 out of 48 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
apps/demos/tsconfig.react-check.json Removed exclusions for FloatingActionButton, VectorMap, and Localization demos now that TS issues are fixed
apps/demos/Demos/VectorMap/**/React/devextreme-vectormap-data.d.ts Added type definitions for GeoJSON world map data (multiple files)
apps/demos/Demos/VectorMap/**/React/data.ts Added type definitions for GDP and population data structures
apps/demos/Demos/VectorMap/**/React/App.tsx Improved types with proper casting and optional chaining
apps/demos/Demos/VectorMap/**/React/TooltipTemplate.tsx Added types for tooltip template components (with issues)
apps/demos/Demos/VectorMap/**/React/PieChartComponent.tsx Added proper prop interfaces for PieChart components
apps/demos/Demos/VectorMap/**/ReactJs/*.js Applied consistent changes to JavaScript versions (return {}, optional chaining)
apps/demos/Demos/FloatingActionButton/Overview/React/data.ts Added comprehensive type definitions for direction configuration
apps/demos/Demos/FloatingActionButton/Overview/React/App.tsx Added proper ref typing and improved type safety
apps/demos/Demos/Localization/UsingGlobalize/React/global.d.ts Added global type declarations for third-party modules

Copilot AI review requested due to automatic review settings January 12, 2026 12:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 36 out of 49 changed files in this pull request and generated 4 comments.

Copy link
Contributor

@pharret31 pharret31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some fixes are required

Copilot AI review requested due to automatic review settings January 12, 2026 13:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 36 out of 49 changed files in this pull request and generated 10 comments.

Copilot AI review requested due to automatic review settings January 12, 2026 15:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 36 out of 49 changed files in this pull request and generated 4 comments.

Copilot AI review requested due to automatic review settings January 12, 2026 16:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 36 out of 49 changed files in this pull request and generated 3 comments.

Copilot AI review requested due to automatic review settings January 12, 2026 17:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 37 out of 50 changed files in this pull request and generated no new comments.

@dmlvr dmlvr merged commit 7cd4b78 into DevExpress:26_1 Jan 13, 2026
96 checks passed
dmlvr added a commit to dmlvr/DevExtreme that referenced this pull request Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants